-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show Icons for Completion Kind #10056
Conversation
And a future step could be to make this configurable by Zed themes |
I will make the icons the same color as the text! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's unlikely we'll merge in this current state, as we would need to design all of these icons to match our current icon set.
Okay. Are there plans for themes to configure icons? |
We are planning on adding the ability for extensions to provide icons, but no timeline on it yet. Right now our primary focus with extensions is making sure all of our current language servers can be expressed using the extension API. Here's the tracking issue for icons in extensions: #8843 |
Maybe this PR could be repurposed and show text with a styled font (muted + colored depending on the completion type) instead of the icons for now? 🤔 Icons could then always be added later and made configurable (maybe the text should be configurable by the user/theme too). 🙂 |
I think we’re a little pressed for real estate here to be showing words of variable lengths. |
That's understandable. 🙂 To be honest I wasn't really thinking about whole words, but more like letters, kind of similar to what for example IntelliJ does (m - method, f - field, v - variable, etc.), but perhaps it would be not clear enough. 👍 |
I'm going to close this for now, as I think we'll want to involve our designer, @/iamnbutler, when he's back from paternity leave to make sure we settle on an approach that aligns with our design direction for Zed. |
Hey! @iamnbutler bump (no pressure) |
@iamnbutler If you have time, can you take a look at this? ^:) |
@iamnbutler bump 🙏 |
Takes influence from the icons here and adds completion kind icons to zed completions. (It looks beautiful!)
Preview with Rust-Analyzer on the Zed codebase: